Skip to content

created maven poms#2

Open
msokolov wants to merge 2 commits intofgeorges:masterfrom
msokolov:master
Open

created maven poms#2
msokolov wants to merge 2 commits intofgeorges:masterfrom
msokolov:master

Conversation

@msokolov
Copy link

Florent - I'm sending you this as an example of the way I would recommend supporting a maven build for this project. The way Adam has set up the packaging project is workable, too, but overly complicated for my tastes. The main difference I see is that he has set up a system of interdependent "pom" files in a so-called "multi-module" project. This has some advantages, since you can have one repository look like one maven project. However, Maven at its heart is about building a single artifact (like a jar, war, etc), so these multi-module projects are always a little difficult to work with. They don't automatically work in most IDEs for example. The way this one is set up is as two completely separate poms. There is of course a dependency between the two projects; to build this:

cd http-client-java
mvn install
cd ../http-client-saxon
mvn install

The "install" target installs an artifact in your local repository so it is available to be resolved as a dependency. Ultimately you will want to have them posted to "maven central" so they can be resolved over the internet, but that's a little more involved of a process that requires registering with oss.sonatype.org, giving them a PK cert, etc.

@adamretter
Copy link

@msokolov I just sent an email to @fgeorges and you (your faultin.net address, I hope that is correct?).

For other readers, I have created a new set of Maven projects for the EXPath modules that I hope iwll be adopted, they can be found here:

https://github.com/adamretter/expath-parent
https://github.com/adamretter/tools-java/tree/maven-attempt2
https://github.com/adamretter/tools-saxon/tree/maven-attempt2
https://github.com/adamretter/expath-pkg-java/tree/maven-attempt2
https://github.com/adamretter/expath-http-client-java/tree/maven-attempt2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments